Merged
Conversation
MichaelSun48
approved these changes
Nov 26, 2025
Comment on lines
+333
to
+336
| - [Main README](../README.md) - General getting started guide | ||
| - [Agentex Backend README](../agentex/README.md) - Backend-specific details | ||
| - [Agentex UI README](../agentex-ui/README.md) - Frontend-specific details | ||
| - [Python SDK](https://github.com/scaleapi/scale-agentex-python) - Agent development examples |
There was a problem hiding this comment.
Broken relative links in Additional Resources
Since WINDOWS.md is at the repository root, these ../ prefixed paths resolve to the parent of the repo root, making all four links broken. They should use relative paths without the ../ prefix.
Suggested change
| - [Main README](../README.md) - General getting started guide | |
| - [Agentex Backend README](../agentex/README.md) - Backend-specific details | |
| - [Agentex UI README](../agentex-ui/README.md) - Frontend-specific details | |
| - [Python SDK](https://github.com/scaleapi/scale-agentex-python) - Agent development examples | |
| - [Main README](README.md) - General getting started guide | |
| - [Agentex Backend README](agentex/README.md) - Backend-specific details | |
| - [Agentex UI README](agentex-ui/README.md) - Frontend-specific details | |
| - [Python SDK](https://github.com/scaleapi/scale-agentex-python) - Agent development examples |
Prompt To Fix With AI
This is a comment left during a code review.
Path: WINDOWS.md
Line: 333-336
Comment:
**Broken relative links in Additional Resources**
Since `WINDOWS.md` is at the repository root, these `../` prefixed paths resolve to the *parent* of the repo root, making all four links broken. They should use relative paths without the `../` prefix.
```suggestion
- [Main README](README.md) - General getting started guide
- [Agentex Backend README](agentex/README.md) - Backend-specific details
- [Agentex UI README](agentex-ui/README.md) - Frontend-specific details
- [Python SDK](https://github.com/scaleapi/scale-agentex-python) - Agent development examples
```
How can I resolve this? If you propose a fix, please make it concise.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Generated code creating windows build.ps1 files as well as updated README's for windows users. I have tested the build.ps1 in both agentex and agentex-ui.
Greptile Summary
This PR adds comprehensive Windows support for the Agentex development workflow by introducing PowerShell
build.ps1scripts that mirror the existing Makefiles, a dedicatedWINDOWS.mdguide, and platform-specific sections across all READMEs.build.ps1scripts (root,agentex/,agentex-ui/) faithfully replicate all Makefile targets with proper error handling and parameterizationWINDOWS.mdprovides a complete end-to-end Windows guide covering prerequisites, setup, command reference, troubleshooting, and IDE configurationmacOS/LinuxvsWindows (PowerShell)section headers and equivalent PowerShell commandsWINDOWS.mdhas broken relative links in the Additional Resources section (../README.mdshould beREADME.mdsince the file is at the repo root)Confidence Score: 4/5
Important Files Changed
../prefix when the file is at repo root).repo-setupandhelpcommands. Clean implementation with proper error handling.Flowchart
%%{init: {'theme': 'neutral'}}%% flowchart TD A["Developer on Windows"] --> B{"Which tool?"} B -->|"Root repo"| C["./build.ps1"] B -->|"Backend"| D["agentex/build.ps1"] B -->|"Frontend"| E["agentex-ui/build.ps1"] C --> C1["repo-setup"] D --> D1["install / install-dev"] D --> D2["dev / dev-stop / dev-wipe"] D --> D3["test / test-unit / test-integration"] D --> D4["migration / apply-migrations"] D --> D5["docker-build / serve-docs"] E --> E1["install / dev"] E --> E2["build / build-and-load"] E --> E3["run / run-detached / stop"] E --> E4["typecheck / lint"] E --> E5["push / build-and-push"]Last reviewed commit: 9a524fa
(5/5) You can turn off certain types of comments like style here!